Skip to content

feat: Added displaying of TOTP values#157

Merged
hazcod merged 1 commit into
hazcod:masterfrom
HQJaTu:totp_display
Jun 7, 2026
Merged

feat: Added displaying of TOTP values#157
hazcod merged 1 commit into
hazcod:masterfrom
HQJaTu:totp_display

Conversation

@HQJaTu
Copy link
Copy Markdown
Contributor

@HQJaTu HQJaTu commented Jun 3, 2026

  • calculate value of TOTP, display value on show-command, not with list-command

@hazcod
Copy link
Copy Markdown
Owner

hazcod commented Jun 4, 2026

Thank you! Let me know once done.

@HQJaTu
Copy link
Copy Markdown
Contributor Author

HQJaTu commented Jun 4, 2026

Thank you! Let me know once done.

Absolutely. I just got the first version working and I think my changes: a) work as I intended, b) don't break anything that worked earlier.
Let me run this for couple days on QA purposes.

@HQJaTu HQJaTu marked this pull request as ready for review June 7, 2026 16:12
@HQJaTu
Copy link
Copy Markdown
Contributor Author

HQJaTu commented Jun 7, 2026

Thank you! Let me know once done.

I've been using this for past couple days and I'm liking this improvement.

  • Internal: Field type will be extracted from vault (actually, was done on previous PR)
    • Bugfix: Fields will be displayed on correct order
  • When using -detailed, based on field type, display relevant information
    • TOTP-code and secret key on TOTP-fields
    • Section will act as a subheader
  • list-command won't display secets, show will
  • All above rules apply for -json also

@hazcod hazcod merged commit 36b13ef into hazcod:master Jun 7, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants